Skip to content

Document the new VcpkgInstalledDirRoot property#566

Open
stinos wants to merge 1 commit intoMicrosoftDocs:mainfrom
stinos:installeddirroot
Open

Document the new VcpkgInstalledDirRoot property#566
stinos wants to merge 1 commit intoMicrosoftDocs:mainfrom
stinos:installeddirroot

Conversation

@stinos
Copy link
Copy Markdown

@stinos stinos commented Feb 3, 2026

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 2462269:

✅ Validation status: passed

File Status Preview URL Details
vcpkg/users/buildsystems/msbuild-integration.md ✅Succeeded View

For more details, please refer to the build report.

Copy link
Copy Markdown
Collaborator

@vicroms vicroms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made some style changes.

Comment thread vcpkg/users/buildsystems/msbuild-integration.md Outdated
Comment thread vcpkg/users/buildsystems/msbuild-integration.md Outdated
Comment thread vcpkg/users/buildsystems/msbuild-integration.md Outdated
Comment thread vcpkg/users/buildsystems/msbuild-integration.md Outdated
Comment thread vcpkg/users/buildsystems/msbuild-integration.md Outdated
Comment thread vcpkg/users/buildsystems/msbuild-integration.md Outdated
@stinos stinos force-pushed the installeddirroot branch from 2462269 to 0644eec Compare May 4, 2026 10:02
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 0644eec:

✅ Validation status: passed

File Status Preview URL Details
vcpkg/users/buildsystems/msbuild-integration.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@stinos
Copy link
Copy Markdown
Author

stinos commented May 4, 2026

Proposed changes make sense so updated and rebased. Except for the proposed change to line 243 because it's not strictly related to this PR and seems more fit for a broader 'improve style/readability' (e.g. perhaps the change was proposed to make the sentence start with a capital but there's at least 1 other sentence which also start with vcpkg).

@stinos stinos requested a review from vicroms May 4, 2026 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants